-
Notifications
You must be signed in to change notification settings - Fork 132
[Dynamic Dashboard] Orders card empty state #11536
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## trunk #11536 +/- ##
============================================
- Coverage 40.24% 40.22% -0.02%
Complexity 5202 5202
============================================
Files 1089 1089
Lines 63462 63487 +25
Branches 8711 8718 +7
============================================
Hits 25539 25539
- Misses 35619 35644 +25
Partials 2304 2304 ☔ View full report in Codecov by Sentry. |
# Conflicts: # WooCommerce/src/main/kotlin/com/woocommerce/android/ui/dashboard/orders/DashboardOrdersCard.kt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @0nko, it works well.
I won't merge to give you a chance to visit this discussion p1716276731997339-slack-C03L1NF1EA3 and decide if we should align with iOS on this or not.
Thanks Hicham, I think we're aligned 👍 |
Part of #11461. Adds the empty state UI to orders card.
To test: